Skip to content

Conversation

HDCharles
Copy link
Contributor

@HDCharles HDCharles commented Jun 15, 2024

Summary: previously we were only doing this in the tests but now we have an eval script to along with generate.py

Test Plan:

python eval.py -q "int4wo-64-gptq"

expected results:

(using meta-llama/Llama-2-7b-chat-hf weights)

Base (bfloat16) | 12.212 |
int8dq | 12.262 |
int8wo | 12.204 |
int4wo-64 | 12.843 |
int4wo-64-GPTQ | 12.489

Reviewers:

Subscribers:

Tasks:

Tags:

Summary: previously we were only doing this in the tests but now we have
an eval script to along with generate.py

Test Plan:

python eval.py -q "int4wo-64-gptq"

Reviewers:

Subscribers:

Tasks:

Tags:
Copy link

pytorch-bot bot commented Jun 15, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/374

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit efb18ce with merge base bc2f8b7 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@HDCharles HDCharles requested a review from msaroufim June 15, 2024 03:40
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 15, 2024
@HDCharles HDCharles requested a review from jerryzh168 June 15, 2024 03:40
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
@@ -0,0 +1,134 @@
# Copyright (c) Meta Platforms, Inc. and affiliates.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! since this is not tested in CI mind just printing the expected output and putting it in the github issue description

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i added the values to the PR description

@HDCharles HDCharles requested a review from msaroufim June 18, 2024 04:30
@msaroufim msaroufim merged commit 69ea59c into main Jun 18, 2024
@msaroufim msaroufim deleted the 072_script_for_eval branch June 18, 2024 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants